Skip to content

Conversation

@grzesiek2201
Copy link

@grzesiek2201 grzesiek2201 commented Feb 12, 2025

  • Modified the encoder working principle to interrupts for smoother and more reliable readings.
  • Added a .stl for a bigger screen mentioned here Front panel mounting holes don't line up with display #2.
  • Added a .stl for a hat for smaller (shorter) encoder, otherwise clicking is not possible.

Contributions

  • Modified sketch_sep25a.ino sketch
  • Added 3D Parts / Bigger screen faceplate / Faceplate-bigger-screen.stl
  • Added 3D Parts / Smaller encoder hat / hat-smaller-encoder.stl

Changes in wiring

None

@NullNullZero
Copy link

Nice code, but could you link the library please? There's none available in Arduino IDE, and there are different versions online. Thx!

@grzesiek2201
Copy link
Author

Of course, my bad! Here's the used library: https://github.com/codingABI/KY040

@UrbanCircles
Copy link
Owner

this looks great - would the source of the library be an issue for some? not familiar with this, and I can see this guy asking for it. should we include the link into the sketch itself, if the library is not easily discovered?

@grzesiek2201
Copy link
Author

How about we include the library .h file itself? Let me modify this pull request to reflect that.

@grzesiek2201
Copy link
Author

Hey @UrbanCircles, the pull request has been modified already.

/** Library version */
#define KY040_VERSION "1.0.1"

#include <arduino.h>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This caused an error for me in Arduino IDE until I changed arduino.h to Arduino.h

hackervole added a commit to hackervole/igor that referenced this pull request Jun 27, 2025
This is based on an upstream PR
(UrbanCircles#5), but I took a slightly
different approach.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants